Skip to content

Update Go SDK version to v1.41.1#9646

Closed
long-nt-tran wants to merge 3 commits intomainfrom
long/go-sdk-pkg-upgrade
Closed

Update Go SDK version to v1.41.1#9646
long-nt-tran wants to merge 3 commits intomainfrom
long/go-sdk-pkg-upgrade

Conversation

@long-nt-tran
Copy link
Copy Markdown
Contributor

@long-nt-tran long-nt-tran commented Mar 24, 2026

What changed?

  1. Updated Go SDK to latest version per https://pkg.go.dev/go.temporal.io/sdk?tab=versions, latest I see today is v1.41.1 from Mar 17 2026
  2. Ran go mod tidy
  3. Some tests were failing due to SDK interface changing, ran cd common/testing/mocksdk && go generate -- generated changes in common/testing/mocksdk/client_mock.go
  4. Fixed Nexus test suite due to breaking changes and deprecated APIs in new SDK -- changes in tests/nexus_workflow_test.go

Why?

To use the latest version

How did you test it?

  • built
  • run locally and tested manually
  • covered by existing tests
  • added new unit test(s)
  • added new functional test(s)

@long-nt-tran long-nt-tran force-pushed the long/go-sdk-pkg-upgrade branch 3 times, most recently from f2f789e to 7f37a24 Compare March 27, 2026 22:14
@long-nt-tran long-nt-tran force-pushed the long/go-sdk-pkg-upgrade branch from 7f37a24 to 5dcfcb9 Compare March 30, 2026 13:47
@long-nt-tran long-nt-tran marked this pull request as ready for review March 30, 2026 14:18
@long-nt-tran long-nt-tran requested review from a team as code owners March 30, 2026 14:18
@long-nt-tran long-nt-tran changed the title Update Go SDK version to latest Update Go SDK version to v1.41.1 Mar 30, 2026
@chaptersix
Copy link
Copy Markdown
Contributor

For OSS repos use forks. We don't want a ton of branches in main repo.

github.com/maruel/panicparse/v2 v2.4.0
github.com/mitchellh/mapstructure v1.5.0
github.com/nexus-rpc/sdk-go v0.5.2-0.20260211051645-26b0b4c584e5
github.com/nexus-rpc/sdk-go v0.6.0
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

github.com/nexus-rpc/sdk-go v0.5.2-0.20260211051645-26b0b4c584e5 this was likely pointing at main. Are you sure the newest release is new that this?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think this is slightly outdated, will run go mod tidy again

@long-nt-tran
Copy link
Copy Markdown
Contributor Author

For OSS repos use forks. We don't want a ton of branches in main repo.

thanks, let me do that. The PR guide I saw mentioned to open the PR on "name/feature" branch against main

@long-nt-tran long-nt-tran deleted the long/go-sdk-pkg-upgrade branch March 31, 2026 14:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants